Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build(guix): add debug symbols for osx #5708

Merged
merged 5 commits into from
Nov 16, 2023

Conversation

PastaPastaPasta
Copy link
Member

@PastaPastaPasta PastaPastaPasta commented Nov 16, 2023

Issue being fixed or feature implemented

Add debug symbols for Darwin

What was done?

Added Darwin debug symbols and combine them as output

How Has This Been Tested?

guix build

Breaking Changes

Please describe any breaking changes your code introduces

Checklist:

Go over all the following points, and put an x in all the boxes that apply.

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation
  • I have assigned this pull request to a milestone (for repository code-owners and collaborators only)

@UdjinM6
Copy link

UdjinM6 commented Nov 16, 2023

pls check https://github.com/UdjinM6/dash/commits/pr5708 + you need to rebase this

@PastaPastaPasta PastaPastaPasta modified the milestones: 20, 20.1 Nov 16, 2023
@PastaPastaPasta PastaPastaPasta changed the title guix: add debug symbols for osx build(guix): add debug symbols for osx Nov 16, 2023
@PastaPastaPasta PastaPastaPasta marked this pull request as ready for review November 16, 2023 18:05
@UdjinM6
Copy link

UdjinM6 commented Nov 16, 2023

Unknown scope "guix" found in pull request title

@PastaPastaPasta
Copy link
Member Author

Unknown scope "guix" found in pull request title

Not important imo; it probably should be a scope

Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK (light ACK, confirmed that dSYM-s are produced)

@PastaPastaPasta PastaPastaPasta merged commit a81310a into dashpay:develop Nov 16, 2023
5 of 8 checks passed
@PastaPastaPasta PastaPastaPasta deleted the guix-osx-debug branch November 16, 2023 18:39
PastaPastaPasta added a commit to PastaPastaPasta/dash that referenced this pull request Nov 16, 2023
## Issue being fixed or feature implemented
Add debug symbols for Darwin

## What was done?
Added Darwin debug symbols and combine them as output

## How Has This Been Tested?
guix build

## Breaking Changes
  _Please describe any breaking changes your code introduces_


## Checklist:
_Go over all the following points, and put an `x` in all the boxes that
apply._
- [x] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have added or updated relevant unit/integration/functional/e2e
tests
- [ ] I have made corresponding changes to the documentation
- [x] I have assigned this pull request to a milestone _(for repository
code-owners and collaborators only)_

---------

Co-authored-by: UdjinM6 <[email protected]>
@UdjinM6 UdjinM6 modified the milestones: 20.1, 20.0.1 Nov 16, 2023
@knst
Copy link
Collaborator

knst commented Mar 21, 2024

Guix builds of debug for osx are always non-deterministic.

Do we need them? maybe to-revert it? due to that non-determinism the file with hashes of artefact differs for different builders and impossible to make a combined gpg signature.

@UdjinM6 @PastaPastaPasta

p.s. bitcoin does never provide debug symbols for osx, I guess due to non-determinism, but I haven't found any reasoning.
see bitcoin#17920

@PastaPastaPasta
Copy link
Member Author

Yes; guix osx debug builds are non-deterministic. However, IMO it is valuable to have and archive these debug symbols so that we can take stack traces from versions and decode them; which are able to be used successfully; but taking the macOS .tar.gz and the debug tar.gz; combining them; and then running the binary. If it crashes; it'll print stack traces :)

The non-determinism is annoying; but it's valuable to keep them around so that we have these symbols. We should instead adjust the script for guis-attest to exclude artifacts which include both osx and debug; this way we have the best of both; able to combine detached sigs for release; but also have the Darwin debug symbols when needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants